Skip to content

feat: add version command#114

Open
gianarb wants to merge 1 commit intomainfrom
feat/add-version-command
Open

feat: add version command#114
gianarb wants to merge 1 commit intomainfrom
feat/add-version-command

Conversation

@gianarb
Copy link
Collaborator

@gianarb gianarb commented Mar 2, 2026

This commit adds kubectl version command to datumctl and it configures
goreleaser to correctly set the required variables.

./dist/datumctl_linux_386_sse2/datumctl version
Client Version: v0.10.1-SNAPSHOT-6d21716
Kustomize Version: v5.7.1
Server Version: v0.0.0-master+$Format:%H$
error: server version error: could not parse pre-release/metadata (-master+$Format:%H$) in version "v0.0.0-master+$Format:%H$"

As you can see kubectl fails as well so it looks something related to
the server itself?

kubectl --kubeconfig .ninja/kubeconfig version
Client Version: v1.34.3
Kustomize Version: v5.7.1
Server Version: v0.0.0-master+$Format:%H$
error: server version error: could not parse pre-release/metadata (-master+$Format:%H$) in version "v0.0.0-master+$Format:%H$"

This commit adds `kubectl version` command to datumctl and it configures
goreleaser to correctly set the required variables.

```
./dist/datumctl_linux_386_sse2/datumctl version
Client Version: v0.10.1-SNAPSHOT-6d21716
Kustomize Version: v5.7.1
Server Version: v0.0.0-master+$Format:%H$
error: server version error: could not parse pre-release/metadata (-master+$Format:%H$) in version "v0.0.0-master+$Format:%H$"
```

As you can see kubectl fails as well so it looks something related to
the server itself?

```
kubectl --kubeconfig .ninja/kubeconfig version
Client Version: v1.34.3
Kustomize Version: v5.7.1
Server Version: v0.0.0-master+$Format:%H$
error: server version error: could not parse pre-release/metadata (-master+$Format:%H$) in version "v0.0.0-master+$Format:%H$"
```
@gianarb gianarb requested a review from scotwells March 2, 2026 15:12
@cla-assistant
Copy link

cla-assistant bot commented Mar 2, 2026

CLA assistant check
All committers have signed the CLA.

@scotwells
Copy link
Contributor

Thanks for reporting @gianarb! Created datum-cloud/milo#511 to chase this down in Milo.

@gianarb gianarb enabled auto-merge March 4, 2026 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants